Skip to content

Comments

Трофимов Павел#55

Open
tgkd wants to merge 2 commits intourfu-2016:masterfrom
tgkd:master
Open

Трофимов Павел#55
tgkd wants to merge 2 commits intourfu-2016:masterfrom
tgkd:master

Conversation

@tgkd
Copy link

@tgkd tgkd commented Oct 31, 2016

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

.container
{
border: 2px double black;
font-size: 18px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Используя свойство font получится уложить 2 следующие строчки в одну

text-align: right;
}

article
Copy link

@ninjagrizzly ninjagrizzly Nov 2, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше использовать селекторы по классам, возможна ситуация, когда на странице есть несколько элементов <article>, которые должны выглядеть по-разному (комментарий относится ко всему коду, а не только к данному селектору)


article em
{
column-span: all;
Copy link

@ninjagrizzly ninjagrizzly Nov 2, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Обрати внимание, это свойство у тебя не срабатывает для <em>.

.second-column,
.third-column
{
float: left;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Свойство float в данной задаче нельзя использовать для разметки страницы


.earth-img
{
width: 95%;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Высоту картинки тоже нужно указывать

<body>
<div class="container">
<header>
<div class="page-name">Ведомости</div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для заголовков есть специальные теги <h1> - <h6>

<div class="page-name">Ведомости</div>
</header>
<hr>
<p class="page-info">Выпуск от 30.10.2016 №10</p>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Дату можно обернуть в специальный тег

<hr>
<p class="page-info">Выпуск от 30.10.2016 №10</p>
<hr>
<div class="page-body">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тег <main>

<hr>
<div class="page-body">
<div class="first-column">
<div class="section">Интересное</div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это тоже заголовок

</article>
</div>
<div class="second-column">
<img class="earth-img" src="https://goo.gl/OuaQf3" alt="Земля">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У тега <img> обязательно указывать непустые атрибуты alt и title

</article>
</div>
<div class="third-column">
<div class="section">Новости спорта</div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И это заголовок

@ninjagrizzly
Copy link

Не соответствует требованиям задачи:

  1. Не нашел вертикального текста
  2. Никак не выделяется первая буква первого параграфа статьи
  3. Нужно использовать нестандартные шрифты, подключив их в css файле

@ninjagrizzly
Copy link

🍅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants